From 88eab303d48549a618cef06da86a58a5c6d73faa Mon Sep 17 00:00:00 2001
From: Aaron Schulz <aschulz@wikimedia.org>
Date: Mon, 16 Mar 2015 19:33:25 -0700
Subject: [PATCH] Fixed timestamp comparison in fixUserRegistration

* Also optimized batching logic slightly

Change-Id: I0cedb5b39f996b7baec067890f230cff51468e4b
---
 maintenance/fixUserRegistration.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/maintenance/fixUserRegistration.php b/maintenance/fixUserRegistration.php
index 0120f353f5..43f4b6c0eb 100644
--- a/maintenance/fixUserRegistration.php
+++ b/maintenance/fixUserRegistration.php
@@ -66,7 +66,7 @@ class FixUserRegistration extends Maintenance {
 					__METHOD__
 				);
 				// Update
-				if ( $timestamp !== false ) {
+				if ( $timestamp !== null ) {
 					$dbw->update(
 						'user',
 						array( 'user_registration' => $timestamp ),
@@ -83,7 +83,7 @@ class FixUserRegistration extends Maintenance {
 			$this->output( "Waiting for slaves..." );
 			wfWaitForSlaves();
 			$this->output( " done.\n" );
-		} while ( $res->numRows() );
+		} while ( $res->numRows() >= $this->mBatchSize );
 	}
 }
 
-- 
2.20.1